Skip to content
This repository has been archived by the owner on Dec 10, 2024. It is now read-only.

Add support for Group Protected Environments #1779

Merged
merged 2 commits into from
Sep 2, 2023

Conversation

beekeep
Copy link
Contributor

@beekeep beekeep commented Aug 21, 2023

Add support for Group Protected Environments, similar to Project Protected Environments.

https://docs.gitlab.com/ee/api/group_protected_environments.html

@beekeep beekeep force-pushed the feat/group-protected-envs branch 4 times, most recently from 48ab504 to efa6c1d Compare August 21, 2023 20:33
Add support for Group Protected Environments, similar to Project Protected Environments.
@beekeep beekeep force-pushed the feat/group-protected-envs branch from efa6c1d to e083b13 Compare August 22, 2023 13:54
@beekeep
Copy link
Contributor Author

beekeep commented Aug 22, 2023

@svanharmelen This is ready for your review.

Copy link
Member

@svanharmelen svanharmelen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not sure if copying over all structs is the best solution here (as most of them are a one-2-one match), but for now I'll take it... Thanks 👍🏻

@svanharmelen svanharmelen merged commit a3c81a7 into xanzy:master Sep 2, 2023
@beekeep
Copy link
Contributor Author

beekeep commented Sep 5, 2023

@svanharmelen Thanks for the review. I had the initial thought of re-using the project protected environment structs, but I wasn't sure if it would be better to create structs specific to groups, in case they do change in projects. In the future, would you prefer re-use if the structs are the same?

@svanharmelen
Copy link
Member

I don't know... Generally I prefer to re-use structs if it makes sense, but in this case I think this is OK.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants